-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CORS 설정 수정 #88
CORS 설정 수정 #88
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋아요 테바 👍
블로그 글도 감사합닌다 ~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테바 글도 쓰고 코드도 고치느랴 수고 많았습니다,,
method에 관한게 궁금해서 코멘트 하나 남겼습니다!
.allowedMethods("*") | ||
.allowedHeaders("*"); | ||
.allowedOrigins("http://localhost:8080") | ||
.allowedMethods("GET", "POST", "DELETE") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OPTION 을 넣지 않아도 괜찮나요?!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
실제 작동 테스트를 해보니 작동이 잘 되더라구요..?
그래도 명시적으로 작성해놓는게 좋으니 수정하겠습니다!
PR의 목적이 무엇인가요?
이슈 ID는 무엇인가요?
설명
질문 혹은 공유 사항 (Optional)